[pull] dev from ethereum:dev#1126
Merged
Merged
Conversation
- Expands text-body override to headers along with strong tags - Fixes /bug-bounty/ MarkdownCards which contain sub-headers that were having text-body-medium applied
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Co-Authored-By: Gemini <gemini@google.com>
Adds fixDuplicateHeadingBlocks to the post-import sanitizer. When a structural change to the English source shifts block layout (e.g. the h1 -> frontmatter.title migration), the pipeline's incremental block-matching can emit a section twice: an anchor-less "ghost" heading immediately followed by the correct same-level heading with a {#anchor}. The new pass removes the ghost block (heading + duplicate prose up to the anchored twin), keeping the canonical anchored version, so future structural changes self-heal instead of shipping duplicates. Code-fence-safe; leaves lone anchor-less headings to syncHeaderIdsWithEnglish. Surfaced by PR #18375 (254 occurrences across 69 files, all 24 languages). Adds 6 unit tests; full sanitizer suite passes (599).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Makes explicit in the intl-review reference and the known-patterns KB that ETHGlossary is authoritative for term translations AND transliteration/calque guidance, with a strict hierarchy: a covered term's per-term script_rule governs; items the glossary does not cover (author names, novel brands) use the script-aware fallback; never infer a default script_rule for an unlisted term. This resolves the te "Mario Havel" false positive (a correctly transliterated non-Latin author name is correct, not a keep-Latin error). Also fixes the /filter example payload key (text -> content; the API requires content), catalogs the new MDX build-breaker variants and the duplicate ghost-heading artifact, and updates stale post_import_sanitize.ts references to intl-sanitizer.ts. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
patch(ui): text-body for all headers inside CardContent
i18n: translation pipeline, common.json (all languages)
intl: ghost-heading sanitizer guard + ETHGlossary source-of-truth docs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )